Skip to content

targets/efinix_ti60: use common HyperRAM integration - #770

Merged
enjoy-digital merged 1 commit into
masterfrom
efinix-ti60-hyperram-integration
Jul 27, 2026
Merged

targets/efinix_ti60: use common HyperRAM integration#770
enjoy-digital merged 1 commit into
masterfrom
efinix-ti60-hyperram-integration

Conversation

@enjoy-digital

Copy link
Copy Markdown
Member

Summary

  • replace the TI60 target-local Wishbone, cache, and HyperRAM plumbing with SoC.add_hyperram()
  • preserve the 32 MB main RAM region and 16 KB L2 cache
  • preserve variable latency, the 2:1 clock ratio, and sys2x_ps input sampling

This removes 32 lines of target-specific integration code.

Dependency

Validation

  • generated the TI60 Efinity project with --with-hyperram --build --no-compile
  • compared the previous and new implementations
    • CSR map and CONFIG_L2_SIZE are unchanged
    • Efinity peripheral XML is unchanged
    • SoC hierarchy, memory map, cache size, and HyperRAM parameters are unchanged

Efinity 2025.1 reports the existing sys2x_ps_pll0_clk:PHASE_SHIFT design-check diagnostic for the 315-degree sampling clock in both the previous and new implementations. This migration does not change that clocking configuration.

Replace the target-local Wishbone slave, L2 cache, and HyperRAM wiring with SoC.add_hyperram(). Preserve the 32MB main RAM mapping, 16KB cache, variable latency, 2:1 clock ratio, and phase-shifted input sampling.\n\nThis depends on LiteX commit 6365b2f6c (soc: add optional L2 cache to HyperRAM integration).
@enjoy-digital

Copy link
Copy Markdown
Member Author

CI note: the PR-event inventory failure is inherited from current master: #768 changed the FireAnt default clock after the generated inventory had been produced. #771 contains the matching inventory refresh. The branch push lint is green, and the TI60 target validation is unaffected; the PR-event lint can be rerun after #771 is merged.

@enjoy-digital
enjoy-digital merged commit f8a6f0c into master Jul 27, 2026
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant